No need to clear GTK_MODULES anymore

Not a variable we care about nowadays.

Spruce up config summary

Show all the options. Nothing to hide.

flatpak: Bundle wayland for now

We need a newer wayland than is available in the runtime
currently, so use the power of flatpak and bundle it.

tooltip: Ignore events with pressed buttons

We dont' want to show tooltips in those cases.

gl renderer: Remove unused viewport member

frame: Use underscores in widgets ids in the docs

They will work in composite widget templates this way.

frame: Remove priv pointer

frame: Stop saving label and child allocation

We don't use them for anything anymore.

Remove gtk_render_icon_surface

As part of removing all the cairo_surace_t usage. There's still
gtk_render_icon for the same purpose that takes a GdkTexture*

widget: Use _set_has_tooltip everywhere

real_set_has_tooltip sounds like it's a vfunc implementation, but it
wasn't. The force parameter was also useless so just remove that.

widget: Use priv pointer directly

widget: Update docs to mention ::draw less

Revert "ci: Enable the a11y test suite"

This reverts commit 968a8e1cf0c5bd21e856204d3aa1f4256d7ff814.

things still fail in the gitlab runner

font chooser: Allow tweaking fractions

This makes some sense, and frac is meant to be set globally.

ci: Enable the a11y test suite

The accessibility test suite only performs internal validation.

tests: Update the expected a11y dump results

Keep in sync with the current tree.

The changes are mostly caused by updates in the internal hierarchy of
composite widgets, and the fact that the order in which the widget tree
is traversed is not exactly stable.

tests: Split out the accessibility dump tests

Instead of having a single massive test running through the a11y
directory, we can split off each individual file into its own unit.

Having individual units has several advantages:

 - units are executed in parallel
 - it's easier to identify the failing units
 - logs for failed units are easier to read

ci: Use a common section for Flatpak jobs

Avoid repeating blobs of YAML manually, and use the proper YAML
mechanism to copy-paste blobs instead.

GskRenderer: Plug a memory leak

flowbox: Remove unnecessary queue_draw call

paned: Remove outdated comment

The cursor is not updated in state_flags_changed these days.

build: Print summary last

gl renderer: Don't use texture ids in glBindFramebuffer calls

snapshot: don't create offset nodes inside offset nodes

It's pretty easy to make that one offset node.

gl renderer: Increase GArray size by 2 up front

Since we know we will add two elements anyway.

widget: Avoid a dead assignment

tooltip: Avoid some possibly uninitialized values

po: fix zh_CN translation

wayland: Don't freeze the frame clock too early

We can't freeze the frame clock on commit, but only after-paint,
otherwise the frameclock will resume in the paint stage.

So freeze the frame clock at the end of the frame if we are waiting for
a frame callback.

Note; The diff is only lage because of indentation changes due to
avoiding early returns in favor of a branch.

gdk: Remove gdk_surface_create_similar_image_surface()

It's unused.

gdk: Remove gdk_cairo_surface_create_from_pixbuf()

It's unused and people should use textures and snapshots anyway.

widget-factory: Use a pixbuf instead of a surface

This gets rid of the last user of
gdk_cairo_surface_create_from_pixbuf().

testgtk: Don't set cairo surfaces as icons

The code expects textures these days, so use those.

gdk: Remove gdk_cairo_get_drawing_context()

It's unused.

tests: Use gdk_texture_save_to_png() where appropriate

texture: Add gdk_texture_save_to_png()

It's needed for debugging Timm's code, so better have it here than
hidden in my random-patch vault.

drawingcontext: Remove unused APIs

ci: allow flatpak jobs to fail for now

We can easily revert this when things are fixed again.

scrolledwindow: Fix indicator revealing

We need to increase the scrollbar bounds size.

gl renderer: Slightly change ops_offset behavior

Add the given delta to the current one instead of replacing it.

gl renderer: Don't draw all rounded clip nodes to a texture

We only need to do that if the rounde clip node intersects with the
outer one.

GDK W32: Fix a missing weak referencing

The changes in a82d67bb7d9290693e8d8c503e65694bb62b44ca didn't
preserve a g_object_weak_ref() call that we need to ensure the
objects in hash map don't become stale. Fix this.

Merge branch 'gitlab-ci-ccache-again' into 'master'

gitlab-ci: Reenable ccache but disable it during the initial meson call

See merge request GNOME/gtk!111

Simplify the Wayland code generation

Instead of going through an ancillary script to strip away the
`WL_EXPORT` annotation from the generated code, we should bump up the
required version of Wayland, and use the `private-code` argument for
wayland-scanner, which does the right thing for us.

gitlab-ci: Reenable ccache but disable it during the initial meson call

There where some problems (??) with ccache not detecting changes during meson
checks. Setting CCACHE_DISABLE during the meson execution makes ccache not use
the cache and pass things directly to the compiler.

gl renderer: Render everyhing according to MV scale

And not the surface's scale factor. This way the magnifier works.

gl renderer: Remove unused field

ci: Update container to Fedora 28

We're going to need updated dependencies that we cannot build as
subprojects, like wayland-protocols.

widget: Not all toplevels are containers

Guard against GtkInvisible.

widget: Only start/stop idle sizers for toplevels

They are the only widgets that have idle sizers.

widget: Move frame clock handling to vfunc

Instead of connecting to / disconnecting from the frame clock, do it
inside the vfuncs next to changing the priv->realized boolean.

This removes a race between those 2 cases that could cause child
widgets' unrealize handlers to reconnect this widget to the frame clock
because it was still marked as realize when the widget had already
disconnected from the frame clock.

Fixes #168

widget: Remove gtk_widget_set_realized()

Everybody chains up now.

gtk: Chain up in toplevels' realize

Don't call set_realized() manually.

widget: Make gtk_widget_real_realize() stricter

But in turn, also allow it to work on widgets with their own surface.

This way, we can chain up from everywhere and won't have to export
gtk_widget_set_realized().

widget: Assert unrealizing

Instead of just setting the flag, assert that it was set.

widget: Remove unused member

container: Remove gtk_container_maybe_start_idle_sizer()

Fold it into gtk_container_start_idle_sizer()

container: Remove gtk_container_queue_resize_handler()

It's the same as gtk_container_maybe_start_idle_sizer()

container: Use maybe_start_idle_sizer()

That way, we don't accidentally start an idle sizer for unrealized
widgets.

window: Don't queue resize handler

It's not needed, because the queue_allocate() call right above it does
this work.

widget: Don't queue resize handler twice

We already queue it when we set_alloc_needed(), so there's no need to do
it again.

container: Don't store the resize clock

Use gtk_widget_get_frame_clock() instead

Merge branch 'gitlab-ci-mingw' into 'master'

gitlab-ci: mingw build

See merge request GNOME/gtk!105

gitlab-ci: Add a 32bit mingw build

Terminate strncpy() buffers correctly

When using strncpy() with a buffer we need to account for the
terminating NUL character. GCC 8 started warning when using PPD_MAX_NAME
as the buffer length for strncpy() because the buffer we're copying into
has the same length — which means that the terminating NUL may be
skipped if the source string has a length of PPD_MAX_NAME.

The appropriate way to handle the case where we're copying a source with
a length bigger than of PPD_MAX_NAME is, as reported in the strncpy()
documentation, to copy `PPD_MAX_NAME - 1` bytes, and explicitly NUL
terminate the destination buffer. This has the additional benefit of
avoiding the compiler warning.

wayland: Factor out frame timings request code

Also call the code everywhere we draw and not just with Cairo.

filechooserbutton: Fix reserved pointers

So gcc stops complaining about unnecessary parenthesis.

ffmpeg: Fix compile warnings for newest version

vulkancontext: Add new error enum

widgetfocus: Use gtk_widget_compute_bounds

Instead of gtk_widget_get_allocation.

widget: Remove _get_own_allocation

Replace all usages of it with _compute_bounds

widget: Remove gtk_widget_get_own_allocation

Use compute_bounds everywhere.

widget: Remove get_own_allocation usages

notebook: Use compute_bounds

popover: Use gtk_widget_compute_bounds

Instead of the private get_own_allocation

toolbar: Avoid a get_own_allocation call

We want the content allocation here since we are allocation child
widgets.

widget: Fix typo in compute_bounds docs

gdk: Protect against negative GDK_SCALE values

Using those should not happen.

GDK W32: use clock after_paint signal for timing updates

* Previous commit had misleading info. The code was
added to begin_paint() instead of end_paint(). Though
that did not affect its performance in any visible way.
* Company advised to move the code to an "after_paint" signal
handler, so that it works on all renderers, not just Cairo.
This change caused high fluctuation in FPS values in fishbowl
when it is put in a situation where it cannot achieve 60fps
(such as using Cairo renderer at ultra-high resolution).
This seems to be deliberate and not a bug.

docs: Link to GResource

Since we're referring to GLib's functionality, might as well save some
time to the reader, and link to the GResource API reference page.

docs/reference/gtk/getting_started: cgit => GitLab

master version of b685eb177932c021975a4b337e37d5911c537de0

GDK W32: set update frequency and timestamp

There is no easily apparent way of being notified when frame updates
happene exactly, so we just query frame info at the end of each paint.
If we query too often (faster than DWM refresh rate), we just get
the same values twice in a row, but that is, hopefully, highly unlikely.

AccelGroup: Remove @See_also of deleted function

gtk_item_factory_new() is not a thing that exists.

Also, maybe the lack of space after the colon was what stopped the link
from rendering.

css: Clipping changes don't need a queue_allocate() anymore

Now that queue_draw() isn't restricted to clip anymore, we don't need to
care about clip in the CSS engine either.

We do keep GTK_CSS_AFFECTS_CLIP around though because GtkWindow does
care for the window's size.

entry: Stop being activatable

Activating entries does not do anything.

Activate the default widget instead.

Merge branch 'master' into 'master'

gdk_x11_display_get_monitor: fix monitor number test logic

See merge request GNOME/gtk!107

gdk_x11_display_get_monitor: fix monitor number test logic

When asked for a nonexistent (positive) monitor number,
gdk_x11_display_get_monitor would (at best) return an uninitialized pointer,
instead of returning NULL.

glcontext: Store the buffer age regions in the GL context

That way, we can store the right region there: The actual painted area
instead of the exposed area (which is way too small).

Also, the GL context is the only user of this data, so storing it there
seems way smarter.

vulkan: Use new resize vfunc to recreate swapchain

... instead of checking sizes for every frame.

drawcontext: Add a surface_resized() vfunc

Call this vfunc whenever the surface's size has changed.

surface: Track all draw contexts created for the surface

widget: Don't do a 0-size check before snapshotting

Even widgets with an empty allocation may still want to draw stuff.
Examples include shadows or child widgets with negative margins.

Fixes GtkEntry's progressbar not showing up anymore.

testmodelbutton: Test :centered, :icon, :inverted

Test more stuff.

testmodelbutton: Fix wrongly overridding MB child

I was setting a GtkButton:label, which resulted in the ModelButton's own
custom children getting lost. That doesn't make for a useful test...

ModelButton: Use WARN_INVALID_PROPERTY, not assert

Do the same thing we do everywhere else for invalid properties.

ModelButton: Fix type Wether => Whether

testmodelbutton: Add explanatory comments to CSS

and load at an earlier, more sensible place, before creating any widgets

ModelButton: Protect against nonsensical gbooleans

the same way we do normally.

tests/testmodelbutton: Add this

and test whether :action-name, :role, and CSS nodes work as expected

https://gitlab.gnome.org/GNOME/gtk/issues/163

Merge branch 'wip/sadiq/fixes' into 'master'

listbox: Fix signal name is documentation

See merge request GNOME/gtk!106